Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Windows build errors : static link wait_set_subscriber_library #349

Closed
wants to merge 1 commit into from

Conversation

adityapande-1995
Copy link
Contributor

@adityapande-1995 adityapande-1995 commented Aug 25, 2022

This PR fixes the problem mentioned here : #344
Windows builds are not able to find wait_set_subscriber_library.lib

Signed-off-by: Aditya [email protected]

@adityapande-1995
Copy link
Contributor Author

Windows CI : Build Status

@@ -36,7 +36,7 @@ ament_target_dependencies(subscriber_not_composable rclcpp std_msgs)
add_executable(subscriber_content_filtering content_filtering.cpp)
ament_target_dependencies(subscriber_content_filtering rclcpp std_msgs)

add_library(wait_set_subscriber_library SHARED
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

rclcpp_components_register_node below is registering this as a plugin. This can only be used with shared libraries. I don't think the problem in #344 is this library being a shared library; instead, I think there's a problem is in whatever is expecting it not to be one.

@clalancette
Copy link
Contributor

This doesn't look the right fix for this issue, so closing this out for now.

@clalancette clalancette closed this Sep 8, 2022
@songyuc
Copy link

songyuc commented Aug 14, 2023

As a beginner in ROS2, I wonder if this problem has been solved yet?

@sloretz sloretz deleted the aditya/fix_windows_build_errors branch August 14, 2023 16:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants